New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cleanup texmanager. #10181
Cleanup texmanager. #10181
Conversation
bfd122c
to
92b1bb4
Compare
get_ps_bbox was last used in 310e707 (2009) |
It looks like |
lib/matplotlib/texmanager.py
Outdated
break | ||
else: | ||
if DEBUG: | ||
print('$s font is not compatible with usetex') | ||
_log.debug('$s font is not compatible with usetex') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is a $s font
? Was this supposed to be a %s
substitution?
lib/matplotlib/texmanager.py
Outdated
|
||
def get_basefile(self, tex, fontsize, dpi=None): | ||
""" | ||
returns a filename based on a hash of the string, fontsize, and dpi | ||
Returns a filename based on a hash of the string, fontsize, and dpi. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since you're changing them all, the first line of the docstring should be imperative (Return, not Returns).
92b1bb4
to
31d8937
Compare
- remove check for "old" cache (the location changed in efa9ea6 (2005)). - postscriptd and pscnt are unused since c0d6110 (2006), deprecate them. - make_ps and make_ps_bbox are unused since 310e707 (2009), deprecate them. - calling \usepackage more than once with the same package does nothing (as a repeated import in Python) so no need to remove calls to \usepackage{type1cm}. - replace DEBUG by _log.debug where appropriate; other cases (forcing file regeneration) can be handled by manually deleting the old file. Various other minor cleanups.
31d8937
to
1d3a628
Compare
all comments are handled |
(as a repeated import in Python) so no need to remove calls to
\usepackage{type1cm}.
file regeneration) can be handled by manually deleting the old file.
Various other minor cleanups.
PR Summary
PR Checklist